Skip to content

Scan status text is truncated in narrow terminal width - #18

Merged
mowi12 merged 2 commits into
mainfrom
fix/17-bug-scan-status-text-is-truncated-in-narrow-terminal-width
Mar 24, 2026
Merged

Scan status text is truncated in narrow terminal width#18
mowi12 merged 2 commits into
mainfrom
fix/17-bug-scan-status-text-is-truncated-in-narrow-terminal-width

Conversation

@mowi12

@mowi12 mowi12 commented Mar 22, 2026

Copy link
Copy Markdown
Owner

Summary

Scan status text is truncated in narrow terminals. To stop this behavior this PR implement a dynamic behavior.
When the width gets to narrow the status text will be displayed in two rows and even shortens if even narrower.

Related Issue

Closes #17

How to Test

  1. Run uv run mr-manager.
  2. Resize the terminal window to a smaller width.
  3. Observe the top scan status area.

Checklist

Either tick the items or cross out those that do not apply (using example text) and provide a brief explanation why they do not apply.

Author

  • I linked the issue that this PR closes.
  • I tested the changes locally.
  • I updated the documentation (if needed).
  • I added/updated tests (if needed).

@mowi12
mowi12 requested a review from Copilot March 22, 2026 17:33
@mowi12 mowi12 self-assigned this Mar 22, 2026
@mowi12 mowi12 added the bug Something isn't working label Mar 22, 2026
@mowi12 mowi12 linked an issue Mar 22, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Improves the Textual UI’s scan status header so it remains readable in narrow terminals by switching to a stacked (two-row) layout and optionally using a compact status text variant.

Changes:

  • Add a “stacked” layout mode for the scan status row in ui.tcss.
  • Track full vs compact scan status strings and re-evaluate layout on terminal resize in ui.py.
  • Minor regex tweak for parsing section headers in config.py.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/mr_manager/ui.tcss Adds .stacked styling to switch the scan status row from horizontal to vertical layout.
src/mr_manager/ui.py Implements responsive scan status text/layout logic and hooks it to resize events.
src/mr_manager/config.py Updates the section header regex (no functional UI impact).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/mr_manager/ui.py Outdated
@mowi12
mowi12 force-pushed the fix/17-bug-scan-status-text-is-truncated-in-narrow-terminal-width branch from 855ac6f to fb55fc3 Compare March 24, 2026 20:23
@mowi12
mowi12 merged commit 245a851 into main Mar 24, 2026
4 checks passed
@mowi12
mowi12 deleted the fix/17-bug-scan-status-text-is-truncated-in-narrow-terminal-width branch March 24, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Scan Status Text Is Truncated in Narrow Terminal Width

2 participants